Skip to content

sysgo: restore portal ProxyAdmin owners after superroot migration#19084

Merged
Inphi merged 1 commit intodevelopfrom
inphi/fix-interop-readiness
Feb 4, 2026
Merged

sysgo: restore portal ProxyAdmin owners after superroot migration#19084
Inphi merged 1 commit intodevelopfrom
inphi/fix-interop-readiness

Conversation

@Inphi
Copy link
Copy Markdown
Contributor

@Inphi Inphi commented Feb 4, 2026

During OPCM super-roots migration we temporarily transfer each OptimismPortal’s ProxyAdmin ownership to the DelegateCallProxy. Restore each portal ProxyAdmin back to the L1PAO after shared admin contracts are reset so both portals report the expected proxyAdminOwner()

With this we can re-enable the TestInteropReadiness acceptance-test

fixes #18616

During OPCM super-roots migration we temporarily transfer each OptimismPortal’s
ProxyAdmin ownership to the DelegateCallProxy. Restore each portal
ProxyAdmin back to the L1PAO after shared admin contracts are reset
so both portals report the expected proxyAdminOwner()

With this we can re-enable the TestInteropReadiness acceptance-test
@codecov
Copy link
Copy Markdown

codecov bot commented Feb 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.7%. Comparing base (6c24c04) to head (6287570).
⚠️ Report is 13 commits behind head on develop.

Additional details and impacted files
@@            Coverage Diff             @@
##           develop   #19084     +/-   ##
==========================================
+ Coverage     76.0%    81.7%   +5.6%     
==========================================
  Files          188      133     -55     
  Lines        10943     6913   -4030     
==========================================
- Hits          8326     5649   -2677     
+ Misses        2473     1264   -1209     
+ Partials       144        0    -144     
Flag Coverage Δ
cannon-go-tests-64 ?
contracts-bedrock-tests 81.7% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 55 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Inphi Inphi marked this pull request as ready for review February 4, 2026 17:32
@Inphi Inphi requested review from a team as code owners February 4, 2026 17:32
@Inphi Inphi requested a review from sebastianst February 4, 2026 17:32
@Inphi Inphi enabled auto-merge February 4, 2026 17:32
@Inphi Inphi requested a review from ajsutton February 4, 2026 17:32
@Inphi Inphi added this pull request to the merge queue Feb 4, 2026
Merged via the queue into develop with commit 66dc4ba Feb 4, 2026
83 checks passed
@Inphi Inphi deleted the inphi/fix-interop-readiness branch February 4, 2026 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enable skipped acceptance tests for interop proofs

2 participants